Skip to content

testrunner: fix clang compile error related to dinit#5297

Merged
firewave merged 5 commits intocppcheck-opensource:mainfrom
pfultz2:fix-compile-error-clang-design-inits
Aug 8, 2023
Merged

testrunner: fix clang compile error related to dinit#5297
firewave merged 5 commits intocppcheck-opensource:mainfrom
pfultz2:fix-compile-error-clang-design-inits

Conversation

@pfultz2
Copy link
Copy Markdown
Contributor

@pfultz2 pfultz2 commented Aug 6, 2023

This fixes the compile error introduced in 5d201c4.

@firewave
Copy link
Copy Markdown
Collaborator

firewave commented Aug 6, 2023

What compile error? Please specify.

Maybe that noexcept workaround can also be removed now.

Comment thread test/helpers.h Outdated
Comment thread test/testprocessexecutor.cpp Outdated
@firewave
Copy link
Copy Markdown
Collaborator

firewave commented Aug 6, 2023

Sorry, I didn't see the previous PR. So it seems that workaround can be removed.

@firewave
Copy link
Copy Markdown
Collaborator

firewave commented Aug 7, 2023

This is really strange that you encounter this. I tried Clang 10 to 18 to see what can be cleaned up but none of those give any errors or warnings with the trunk.

Comment thread test/helpers.h
@firewave firewave changed the title Fix compile error in clang testrunner: fix clang compile error related to dinit Aug 8, 2023
@firewave firewave merged commit 037bed5 into cppcheck-opensource:main Aug 8, 2023
@pfultz2 pfultz2 deleted the fix-compile-error-clang-design-inits branch August 9, 2023 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants